data: add SPL token standard to remaining Solana-supporting wallet offers - #3681
data: add SPL token standard to remaining Solana-supporting wallet offers#3681foxxx009 wants to merge 1 commit into
Conversation
Summary
|
USS-Supervisor
left a comment
There was a problem hiding this comment.
Verdict: APPROVE
Risk: LOW
Summary: Required validation is passing on the current head and the reviewer-owned current-cycle link-check run 34060772131 completed successfully. I checked the changed CSV surface and confirmed references/offers/wallets.csv still parses as 106 rows with 22 columns, with unchanged header, row count, and slug set; only the tokensSupport cells for coolwallet, dcent, and gate changed, and all three now include SPL without duplicate slugs or row-width shifts.
Findings: None.
Confidence: HIGH
The source check also supports the change: the touched canonical offers are referenced from listings/specific-networks/solana/wallets.csv, and the linked provider/Gate documentation supports SPL/Solana coverage for the three wallets. The current-cycle link-check summary reported 4 total/unique links, 4 successful, 0 errors, 0 timeouts, and 0 unsupported URLs.
…3692) Third batch of the SPL backfill for Solana-supporting canonical wallet rows not covered by #3669/#3681, based on primary provider documentation: - okxwallet: OKX Learn pages document OKX Wallet SPL Token Support ('OKX Wallet handles both Ethereum and Solana'; support table lists 'SPL Token Support: Yes'); Solana Compass: 'hold SOL and SPL tokens' - tangem: tangem.com blog 'Token compatibility: ERC-20, BEP-20, SPL (Solana), TRC-20'; help center lists Solana among custom-token networks - zerion: zerion.io homepage ('Transaction history across EVM and Solana'; 'integrated Solana'); Google Play listing documents Solana SPL token support (USDT, USDC, BONK, JUP, RAY, PYTH on Solana) Same root cause as #3660/#3669/#3681: these rows are referenced by listings/specific-networks/solana/wallets.csv via !offer:<slug>. No other columns, rows, or files touched. Validation (json-tools validate_csv, csv_to_json, validate) passes.
…fers (#3694) * data: add SPL token standard to remaining Solana-supporting wallet offers Fourth batch of the SPL backfill for Solana-supporting canonical wallet offers, not covered by #3669, #3681 or #3692: - `1inch` - `atomic-wallet` - `coin-wallet` - `cypherock-x1` - `ecoin-wallet` - `frontier` - `fxwallet` - `gem-wallet` - `holdstation` - `keystone-3-pro` - `talisman` - `trezor` - `zengo` - `zengo-pro` - `zypto` 15 cells in `references/offers/wallets.csv`. No other columns, rows, or files touched. ## Evidence (primary provider docs) | Wallet | Source | Quote | |---|---|---| | `1inch` | 1inch.io/wallet | Solana added to 1inch Wallet (2025-06-30); supports SOL and SPL tokens | | `atomic-wallet` | atomicwallet.io + Solana Compass | Browser extension ships native Solana support: SOL, SPL tokens and NFTs | | `coin-wallet` | coin.space/solana-wallet | Official Solana wallet page lists SOL and SPL token support | | `cypherock-x1` | Cypherock release notes | Firmware v0.6.1793 adds SPL token support | | `ecoin-wallet` | ecoinwallet.org/solana-wallet | v1.7.1 Solana integration with SPL token support | | `frontier` | frontier.xyz | Solana listed among supported networks (35+ chains) | | `fxwallet` | fxwallet.com + Solana Compass | Native support for SOL and SPL tokens | | `gem-wallet` | docs.gemwallet.com | Full SPL and SPL Token-2022 support | | `holdstation` | docs.holdstation.com + App Store v5.7 | "Solana Support - manage Solana (SOL) and SPL tokens"; Solana listed as Non-EVM | | `keystone-3-pro` | Solana Compass | Manage SOL, SPL tokens and NFTs; Solflare integration | | `talisman` | docs.talisman.xyz | "Solana, Substrate, and Bittensor networks and token balances are automatically enabled" | | `trezor` | trezor.io | Solana and SPL support; Suite adds Token-2022 (2025-01) and native staking (2025-03) | | `zengo` | zengo.com | "Send and receive SOL and SPL tokens" | | `zengo-pro` | zengo.com/pro | Pro tier shares the same engine; Solana/SPL support | | `zypto` | zypto.com | Full SOL and SPL token support with Raydium DEX integration | Same root cause as the Phantom correction in #3660 and the #3669/#3681/#3692 backfills: these canonical rows are referenced by `listings/specific-networks/solana/wallets.csv` via `!offer:<slug>`, so the missing SPL standard propagates as inherited data to every network listing referencing these wallets. ## Deliberately not included - `haha-wallet`: vendor copy is contradictory (App Store states "Ethereum and Polygon networks supported" while also listing Solana), so it needs a clearer primary source before claiming SPL. - `metamask`, `base-wallet`, `ctrl-wallet`, `wigwam`, `pillar`, `clave-wallet`: still under verification. - `kuma-wallet`, `taho`: these do not support Solana (EVM / Polkadot-ecosystem only) and appear mis-listed on the Solana page; reported separately instead of being given SPL. ## Validation - Surgical edits verified programmatically: 15 changed cells, 0 off-target cells. - Row count unchanged at 106; only the `tokensSupport` column was touched. Rewards address: 0x2409b47a530be3831158f10b08ac93f7d08c1ff2 * data: add SPL token standard to MetaMask offer MetaMask ships native Solana support, so its canonical row needs the SPL token standard alongside its EVM standards. Evidence: - MetaMask Support, "Navigate Solana in MetaMask": "MetaMask supports Solana, allowing you to manage your SOL and SPL tokens alongside your EVM-supported (Ethereum) assets." - Consensys blog, "Native Solana support in MetaMask accelerates new era of multichain access": users can "seamlessly manage Solana and SPL tokens, including sending, receiving, swapping, bridging, and onramping assets". 1 cell in `references/offers/wallets.csv`. No other columns, rows, or files touched. Rewards address: 0x2409b47a530be3831158f10b08ac93f7d08c1ff2 * data: add SPL token standard to Wigwam and Base (Coinbase Wallet) offers Both wallets ship Solana support, so their canonical rows need the SPL token standard alongside their EVM standards. Evidence: - Wigwam (official, https://wigwam.app): "Wigwam is a free crypto wallet that supports the most common EVM blockchains and Solana from the box"; FAQ: "The Wigwam app supports all tokens issued on EVM blockchains and Solana." - Coinbase (official blog, Solana support announcement): "Coinbase Wallet extension users can store, send, and receive Solana (SOL) and all of its SPL tokens alongside all of their EVM-compatible assets" -- Coinbase Wallet is the product now shipped as Base App / Base Wallet. 2 cells in `references/offers/wallets.csv`. No other columns, rows, or files touched. Rewards address: 0x2409b47a530be3831158f10b08ac93f7d08c1ff2 * remove spl from trezor * remove duplicated language --------- Co-authored-by: Yevhenii Kobets <eu.kobets@gmail.com>
…oolwallet, dcent, gate) Rewards address: 0x2409b47a530be3831158f10b08ac93f7d08c1ff2
80c1d32 to
2ccc446
Compare
Summary
Backfills the missing
SPLtoken standard in the canonicaltokensSupportcell for the three Solana-supporting wallet offers not covered by #3669:coolwalletdcentgate3 cells in
references/offers/wallets.csv. No other columns, rows, or files touched.Evidence (primary provider docs)
coolwalletdcentgateSame root cause as the Phantom correction in #3660 and the #3669 backfill: these canonical rows are referenced by
listings/specific-networks/solana/wallets.csvvia!offer:<slug>, so the missing SPL standard propagates as inherited data to every network listing referencing these wallets.Validation
validate_csv.py: All checks passed.csv_to_json.py+validate.py(json-tools @ origin/json-tools, schema 2.0.0): exit 0.Rewards address: 0x2409b47a530be3831158f10b08ac93f7d08c1ff2